Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AUD-39] 인포창 바깥쪽 클릭 시 인포창 사라지는 기능 구현 #26

Merged
merged 5 commits into from
Jan 30, 2024

Conversation

KimGaeun0806
Copy link
Member

📃 변경사항

  • 이전 PR의 코멘트들 관련한 수정사항
    • 파라미터로 받아오는 event 객체를 e에서event로 변경
    • TmapModule에서 타입 추론 가능한 변수들의 타입 명시하는 부분 제거
    • Marker 컴포넌트의 number 파라미터를 order로 이름 변경
  • TmapModule에서 infoWindow 객체를 담는 infoWindows배열을 하나의 객체만 저장하는 형태로 변경
  • createInfoWindow에서 기존에는 바로 removeInfoWindow를 실행했는데, infoWindow 변수가 null이 아닐 때만 removeInfoWindow를 실행시키도록 변경
  • handleMapClick 함수에서 infoWindow가 null이 아닐 경우에 infoWindow를 null로 바꾸고 return하도록 수정 (주요 변경사항)
  • removeInfoWindow 메서드에서 setMap(null) 이외에 infoWindow 변수 자체를 null로 바꾸는 로직 추기

🎇 동작 화면

2024-01-30.2.29.37.mov

@KimGaeun0806 KimGaeun0806 added the ✨ Feature 기능 개발 label Jan 30, 2024
@KimGaeun0806 KimGaeun0806 self-assigned this Jan 30, 2024
Copy link
Collaborator

@RookieAND RookieAND left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

infoWindow 의 유무에 따라서 윈도우를 지울지 말지를 결정하는 분기 로직의 추가군요.
리뷰 변경 사항 반영 및 새로운 기능 추가도 모두 확인했습니다. 고생하셨습니다.

@KimGaeun0806 KimGaeun0806 merged commit bd57d14 into develop Jan 30, 2024
1 check passed
@KimGaeun0806 KimGaeun0806 deleted the feature/AUD-39 branch January 30, 2024 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants